Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added check on number of offline ingestors and relevant configuration #100

Merged
merged 5 commits into from
Jan 14, 2025

Conversation

nitrosx
Copy link
Contributor

@nitrosx nitrosx commented Jan 9, 2025

This PR updates the sample configuration and adds the checks to limit the number of offline ingestors running.
The number of offline ingestors and the wait time for the next check have been added to the configuration

@nitrosx nitrosx requested a review from YooSunYoung January 9, 2025 16:14
@nitrosx nitrosx self-assigned this Jan 9, 2025
@nitrosx nitrosx added bug Something isn't working enhancement New feature or request labels Jan 9, 2025
@YooSunYoung YooSunYoung changed the base branch from fix-offline-command to fix-offline-command-patch January 14, 2025 09:05
Copy link
Contributor

@YooSunYoung YooSunYoung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nitrosx I just pushed fixes and it looks fine now...! You can merge it after reviewing #101 and #100

Base automatically changed from fix-offline-command-patch to fix-offline-command January 14, 2025 14:11
@nitrosx nitrosx merged commit e58d805 into fix-offline-command Jan 14, 2025
3 checks passed
@nitrosx nitrosx deleted the production-testing-20250109 branch January 14, 2025 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants